Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4기 황준호]JPA : Mission 3 제출입니다. #303

Open
wants to merge 1 commit into
base: juno-junho
Choose a base branch
from

Conversation

juno-junho
Copy link

📌 과제 설명

Order, order_item, item의 연관관계 매핑을 실습해본다.

👩‍💻 요구 사항과 구현 내용

  • 연관관계를 학습하고 알맞게 설정하였습니다.

✅ PR 포인트 & 궁금한 점

cascade의 remove와 orphanRemoval = true의 차이점이 무엇일까요?
블로그 링크를 통해서 찾아 보았는데 잘 이해가 가지 않네요.
영한님 강의에서는 연관관계 편의 메서드 (값 세팅하는 메서드)는 둘 중 하나 정해서 생성하라고 하는데, fk가 있는 곳을 정해 세팅하는것이 맞을까요?

Copy link

@hyeon-z hyeon-z left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

강의에서 배운 거를 야무지게 사용해서 하셨군요!☺️


import java.time.LocalDateTime;

@Getter
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ Getter 따로 안쓰이고 있는 것 같은데 삭제하면 좋을 것 같습니다!

Comment on lines +20 to +21
private int price;
private int stockQuantity;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

엔터가 빠져 있습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants